-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Derive multisig ownership history based on past transactions and latest discovery output #103
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
(ownerModification.decoded?.method === 'addOwnerWithThreshold' || | ||
ownerModification.decoded?.method === 'removeOwner') && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we also need the function changeThreshold
here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We ain't need threshold since whenever transactions was successfully executed, we assume quorum has been reached
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're only recreating owner's amount history
const ownerChangingTransactions = allTransactions | ||
.map((tx) => ({ raw: tx, decoded: getDecodedProperties(tx) })) | ||
.filter( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we sure that all the transactions here have been executed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's exactly the purpose of line ~153, where we filter only executed transactions
ae75767
to
8593445
Compare
8593445
to
5aa2111
Compare
Resolves L2B-3124
What's changed
Ownership history is derived from internal multi-sig transactions.
I reorganized a few things, but nothing breaking was introduced.
Notes
We could fire
getDecodedProperties
only once, now we do that in2 * amount of txs
manner. We ain't need optimizations for now